Skip to content

fix: update status window colors in dark mode#5773

Merged
walterbender merged 1 commit intosugarlabs:masterfrom
kh-ub-ayb:status-darkmode-fix
Feb 27, 2026
Merged

fix: update status window colors in dark mode#5773
walterbender merged 1 commit intosugarlabs:masterfrom
kh-ub-ayb:status-darkmode-fix

Conversation

@kh-ub-ayb
Copy link
Contributor

@kh-ub-ayb kh-ub-ayb commented Feb 17, 2026

Description

This PR fixes display issues in the STATUS window when Dark Mode is enabled.


Changes

  • Updated css/darkmode.css
    • Set td.headcol background to dark grey (#363636).
    • Set other td elements to dark blue (#022363) .
    • Changed text and bold colors to white for better readability.
    • Used !important to ensure these styles correctly override any inline background colors set by the widget's JavaScript logic

Before

Screenshot 2026-02-17 193032

After

Screenshot 2026-02-17 195700

Verification

All the changes have been tested locally .

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@subhraneel2005
Copy link
Contributor

hi @kh-ub-ayb ,

i have tested the changes locally and its working as u mentioned, good work

Copy link
Contributor

@vanshika2720 vanshika2720 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kh-ub-ayb Good improvement to dark mode readability
Please review usage of !important to ensure it’s necessary.
Otherwise, approved.

@kh-ub-ayb
Copy link
Contributor Author

@kh-ub-ayb Good improvement to dark mode readability Please review usage of !important to ensure it’s necessary. Otherwise, approved.

I used !important because the widget sets inline background styles , and this was needed to properly override them in dark mode.

@kh-ub-ayb
Copy link
Contributor Author

@walterbender @omsuneri please review

@walterbender walterbender merged commit 9ae0c1b into sugarlabs:master Feb 27, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants